Update tpo.cc (#643)

Some .tpo file tracks were/are parsed such that wild jumps occur in lat/lon, creating invalid coords which were not caught by output routines. Some logic flaws and int size errors have been fixed, but some tracks still contain invalid points. Tracks not corrected by this patch now result in warning messages to STDOUT. Entirely valid tracks will also generate warnings if lat or lon changes by a full degree between adjacent track points (should never happen). Non-fatal warning allows manual removal of the few bad track points  in an output format like GPX. Damage is apparently limited to half a dozen incremental points before the next full point, but that can't be guaranteed.

Co-authored-by: tsteven4 <13596209+tsteven4@users.noreply.github.com>

don't set gpsbabel::XmlStreamWriter codec in mapfactor (#733)

We use utf-8 by default as of #671.

Merge pull request #732 from tsteven4/qtprep103

Qt 6 compatiblity changes

avoid std::min with QString::size

missed one in previous commit.

don't include QCharRef.

missed one in previous commit.

Merge pull request #731 from tsteven4/qt6prep102

compatible changes for Qt6

don't include QByteRef.

It is gone in Qt6.  It was not documented as a class in Qt5.
In Qt5 QByteRef just included qbytearray.h.

don't include QCharRef.

It is gone in Qt6.  It was a "helper class for QString".  It was not
documented as a class in Qt5.  In Qt5 QCharRef just included qstring.h.

don't include QStaticStringData.

missed one in the previous commit.

Merge pull request #730 from tsteven4/qt6prep101

compatible changes for Qt6

accomodate the deletion of QDateTime::toTime_t in Qt6.

don't include QStaticStringData.

It goes away in Qt6.  Besides, the documented include is QString,
which in Qt 5 will include qstringliteral.h just as QStaticStringData
does.

avoid std::min with QString::size, which

changes types in Qt6.

migrate from QRegExp to QRegularExpression. (#729)

compatible fixes for qt6 warnings and conflicts. (#728)

* quite qt6 yammering about foreach.

* avoid name collision with Qt6.

* fix warning: type qualifiers ignored on cast result type

Don't use Qt module names in include directives. (#727)

This simplifies migration to Qt6 where some classes moved.

This was done mostly by an ed script, although mkstyle.sh took some
hand holding.

1,$g/#include <QtCore\// s/<QtCore\/\([^>]*\)>/<\1>       /
1,$g/#include <QtGui\// s/<QtGui\/\([^>]*\)>/<\1>      /
1,$g/#include <QtNetwork\// s/<QtNetwork\/\([^>]*\)>/<\1>          /
1,$g/#include <QtWidgets\// s/<QtWidgets\/\([^>]*\)>/<\1>          /
1,$g/ *$/ s/ *$//
w
q

more qstringview work for qt6. (#726)

update gui/gpx.cc to use QStringView for Qt6. (#725)

pass QStringViews by value, not const reference. (#724)

As recommended in the QStringView documentation.

fixes for #722, Mechanically remove QStringRef use for QStringView to help with Qt6. (#723)

* Fix typo in maggeo from search-and-replace work earlier.

* Revert startOfDate() calls to keep floor at 5.12.x to keep builders happy. :-/

* Explictly get QString from StringView in gpx for compat with older Qt builds.

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

Mechanically remove QStringRef use for QStringView to help with Qt6. (#722)

* Make more string literals encoded at UTF-16 (bloaty-bytes) to
reduce conversions.
* Favor QStringView(Qt6-ism that mirrors C++ standard std::stringview)
over QStringRef (Qt5 hack).

Co-authored-by: Robert Lipe <robertlipe@gmail.com>

don't limit simplify count maximum in GUI (#721)

Remove Bushnell and Bushnell_trl formats.
These were added in 2009 when Onix unit were on firesale at places like Woot
as Bushnell had just exited the handheld GPS space. There was a flurry of
activity by a low number of users, followed by years of silence beyond
maintenance burden.

fix map preview bug that dropped tracks with 2 pts. (#714)

add fedora 34 to CI (#713)

Minorfixes to "new" gtm_stringread() functions to eliminate leading and trailing spaces (#710)

* Trim more uses of C date and potentially unsafe memory practices.

* Add warning fixes for garmin_xt and exif

Merge pull request #664 from GPSBabel/reaper

Remove dead formats: alan, an1, netstumbler, naviguide.

restore lost format options when using positional parameters (#709)

* restore format options with positional parameters.

* fix positional test

Merge pull request #707 from GPSBabel/pvs

PVS-Studio fixes

Bring back some of the files I nuked and move them to deprecated/

fix some format mismatches found by PVS-Studio

V576 Incorrect format. Consider checking the second actual argument of the 'asprintf' function. The SIGNED integer type argument is expected.

QString::asprintf is not recommended for new code anyway.

fix erroneous Qt translation caught by PVS studio.

V646 Consider inspecting the application's logic. It's possible that 'else' keyword is missing.

This makes me wonder if this code is ever used.  If args == "OutputFromUnit"
it would throw a fatal error since b9cdbe4e8dc02cff07af9e0fa28adb2372337963 Oct 22, 2013

fix sequence related undefined behavior found by PVS Studio.

V567 The modification of the 's' variable is unsequenced relative to another operation on the same variable. This may lead to undefined behavior.

pvs fix

V668 There is no sense in testing the 'wpt_tmp' pointer against null, as the memory was allocated using the 'new' operator. The exception will be generated in the case of memory allocation error.

fix another pvs warning

V758 The 'pos' reference becomes invalid when temporary object returned by a function is destroyed.

fix ancient memory leak in filter_vecs. (#706)

another pvs studio find

↑ V547 Expression is always false.

PVS Studio kills an old bug.

V1064 The '360' operand of integer division is less than the '65535' one. The result will always be zero.

Also, we need to set the flag for course.

make some Qt containers const. (#705)

This avoids checks for detachment.

fix Qt6 deprecation warning in mapbar_track. (#703)

* fix Qt6 deprecation warning in mapbar_track.

"warning: 'fromUtf16' is deprecated: Use char16_t* overload."
However, it is recommended to use QString(const QChar *, int) or
QString(const QChar *) instead of
QString::fromUtf16(const ushort *unicode, int size = -1) and
QString::fromUtf16(const char16_t *str, int size = -1) anyway.

* fix Either the condition 'track==nullptr' is redundant or there is possible null pointer dereference.

as this isn't using a non-throwing allocation function memory allocation
errors will cause std::bad_alloc to be thrown anyway, we won't return
with track == nullptr.

* eliminate another non-functional memory check.

fix Wmissing-field-initializers. (#704)

These appear with Qt6 on macos which enables Wextra.

use default parameter instead of QString::KeepEmptyParts, (#702)

simplifying migration to Qt6.

Parse empty gx:coord elements in KML reader (#701)

Add test for empty gx:coord elements

modernize nmea reader (#700)

use object tree to cleanup UpgradeCheck, UpgradeCheck::manager_ and (#692)

UpgradeCheck::replyId.

Introduce RunMachine to manage gpsbabel process. (#699)

* Introduce RunMachine to manage gpsbabel process.

* fix comment

* guard runmachine init state

fix gui usage of duplicate filter. (#695)

the shortname and location options are not exclusive.

this resolves #686.

remove a few old kludges that are no longer needed. (#691)

* remove a few old kludges that are no longer needed.

* clean up upgrade a bit

"GPSBabel Version " doesn't need to be removed from currentVersionIn
in UpgradeCheck::checkForUpgrade,
that is taken care of in MainWindow::findBabelVersion

update readme.md with current continuous integration provider (#672)

* update README for new CI.

* use list for workflows

* appease codacy list item indent

* restore lost newline

* don't mention pre-releases in README.md

remove leftover includes from autotools (#689)

* remove leftovers from autotools

we no longer have config.h, HAVE_CONFIG_H is always false.

* more config.h deletions (from jeeps)

make src/core and jeeps includes relative to top cli directory (#688)

* make jeeps includes relative to cli diretory.

* make src/core includes all relative to top level src dir.

prefer implicitly-defined dtors instead of user dtors with empty bodies. (#685)

Not only is this simpler, it avoids clang-tidy complaining about
warning: class * defines a non-default destructor but does not define a copy
constructor, a copy assignment operator, a move constructor or a move
assignment operator [cppcoreguidelines-special-member-functions]

cleanup FormatLoad class header file. (#684)

clang-tidy modernize-use-default-member-init

This allows the use of the implicitly-defined default ctor.
Use implicitly-defined dtor as well.

further cleanups for Format class header. (#683)

manual fix for typedef found by clang-tidy modernize-use-using

clazy fixes for clazy-function-args-by-ref

clang-tidy fix for modernize-use-default-member-init

use new style connect, i.e. function pointers (#682)

* migrate from old style connects to the new style.

The changes using qOverload were done by hand, the rest were done
with clazy 1.9.

The ones using the TreeAction constructor were not found by clazy
and would require more manual intervention.

Also, I note uic generates old style connects.

* eliminate our TreeAction class in favor of QMenu::addAction method.

It is a mystery to me why we created TreeAction.  The QMenu method existed
in Qt 4.3.  Documentation before 4.3 is scarce.
QMenu::addAction(const QString &text, const QObject *receiver, const char *member, const QKeySequence &shortcut = 0)

* clazy updates gmapdlg connects.

* add TODOs for Qt6 related to qOverload.

drop support for QtWebKit. (#680)

Our QtWebKit build didn't work any longer on Ubuntu bionic or focal,
but went into an infinite loop if map preview was used.
I observed this behavior with builds of 1.7.0 as well.

Document exisiting gui configuration option disable-mappreview.

I note that latest fedora build spec falls back to QtWebKit on some
architectures where QtWebEngine is not available.  This change will
require them to fall back to disabling the map preview instead.
https://src.fedoraproject.org/rpms/gpsbabel/blob/rawhide/f/gpsbabel.spec

QtWebEngine appears to be coming to Qt6 with or after 6.2.
https://bugreports.qt.io/browse/QTBUG-63235

delete FAKE_LANGUAGE_MENU code. (#681)

This code hadn't compiled for a while, and is no longer necessary
to test translations.

replace obsolete and overloaded QProcess signal error(QProcess::ProcessError error) (#679)

with the recommended replacement, QProcess signal errorOccurred(QProcess::ProcessError error).
It seems this should have caused an issue in Qt6, but we hadn't noticed it yet.

fix deprecated-copy waring. (#675)

This fixes
"formatload.cc:127:40: warning: implicitly-declared
 ‘Format& Format::operator=(const Format&)’ is deprecated [-Wdeprecated-copy]"

Use default member initializers for FormatOption and Format classes.
This allows the use of the default constructor and simplifies the
creation of parameterized constructors.

Use implicit copy constructor for FormatOption and Format classes.
Note the previous copy constructor for the Format class was not
really a copy constructor, it re-initialized readUseCount and writeUseCount.
This was unnecessary.  The only place we want to use the copy constructor is
in FormatLoad::getFormats and the instances to be copied have just been
created with one of the Format constructors so the use counts will be zero.
There are plenty of other opportunities where Qt might detach formatList_
and use the copy constructor.  Most of these cases are read accesses and
improved const correctness could eliminate the possiblity, i.e. using
QList::at instead of QList::operator[].  A few of these are modifying an
item on the formatList_, and Qt will insist the copy constructor is available
at compile time even if it is never used at run time.

refactoring for cppcheck "The scope of the variable can be reduced" (#677)

* refactoring for cppcheck "The scope of the variable can be reduced"

The c++17 "If Statement with Initializer" often are useful in resovling
these.

* more scope reduction.

* and another.

Merge pull request #676 from tsteven4/moretidy

clang tidy fixes

qt6 compatibility changes. (#674)

QDateTime::toTime_t has been obsolete for a long time, it is gone
in Qt6.

tidy additional readability-convert-member-functions-to-static

tidy modernize-use-auto

tidy modernize-deprecated-headers

Merge pull request #673 from tsteven4/tidyupagain

clang tidy readability fixes

tidy readability-qualified-auto

I also ran these through astyle with our astylerc file to match
our standard.  This resulted in significant reformatting in geojson.cc

tidy readability-const-return-type

with a little whitespace cleanup by hand and removal of an
unused deprecated function.

tidy readability-convert-member-functions-to-static

this took a little bit of hand editing for whitespace and
one change that had to be reverted wrt a Functor signature.

trust Qt to strip out characters illegal in xml. (#671)

this was added in Qt 5.11.0.

clang tidy readability-make-member-function-const

Update standards to c++17, Qt >= 5.12, MSVC >= 2017 (#670)

* raise minimum standards to c++17, Qt 5.12, MSVC 2017.

* avoid std::optional::value which didn't work until macOS 10.14.

* avoid std::optional::value which didn't work until macOS 10.14.

* update runner for codeql so we have Qt >= 5.12

* try harder to get codeql working with newer Qt.

* use c++17 fallthrough attribute.

* use c++17 'if constexpr' as suggested by resharper.

* catch mkicondoc up with c++17

cleanup CMakeLists.txt a bit. (#669)

add support for cmake, ninja to Docker images.
get git to ignore various generated files.

fix some codeql warnings. (#663)

* fix some codeql warnings.

* use c++11 z length modifier in format strings with size_t format arguments.

* don't attempt to upload coverage if token is null or not set.

with the github action flow the CODACY_PROJECT_TOKEN is set and null
for foreign pull requests that don't have access to the secrets.

* double protect token

add github actions for CI. (#662)

* add github actions for CI.

* fix shellcheck issues in ci_install_windows.sh

* generalize windows ci build support

cleanup powershell scripts with Invoke-ScriptAnalyzer, including use of
"-Settings CodeFormatting".

create staging directory for windows installer creation.
It is difficult to exclude all the files in the build directory that we don't
want to distribute as the files are a function of the build system.

add windows cmake build.

catch windows installer up with renamed vc redistributable installer. (#667)

catch cmake up (#666)

with recent changes to *.pro files, and
some lost changes to get windows rc files working.

Remove dead formats: alan, an1, netstumbler, naviguide.

As discussed in https://github.com/gpsbabel/gpsbabel/issues/659

Merge pull request #660 from gpsbabel/win64_only

switch windows release build from 32 bit to 64 bit (only).

Merge pull request #658 from gpsbabel/date_demo

Modernize time handling in compegps, globalsat, hiketech, jtr, magpro…

Merge pull request #657 from gpsbabel/improve-errors

Improve errors by displaying full filename paths on open failures.

turn off selected msvc warnings for c compiler.

switch windows release build from 32 bit to 64 bit (only).

Modernize time handling in compegps, globalsat, hiketech, jtr, magproto, util.

Unfix warning in util.cc since it breaks builds

...on Windows and I don't feel like wrangling in an upgrade.

Fix a dumb warning in garmin_txt.cc.

Use full path name when displaying errors opening files for reads and writes.

This doesn't work for every place where we issue fatal errors, but it gets
the high value formats.

Merge pull request #647 from gpsbabel/robertlipe-patch-1

ozi: don't deinit a write stream if we don't have one.

add ozi track write test cases

backward compatible changes for Qt6 (#656)

* Avoid implicit conversion of int to QChar.

In Qt6 the constructor QChar(int code) is explicit by default.

* update codacy coverage reporter

* move coverage generation to focal.

* use binary codacy coverage reporter

move Qt 5.15 build to 5.15.2. (#652)

update mac, linux docker qtio builds to 5.12.10. (#650)

* update mac Qt 5.12 build to 5.12.10.

* update qtio docker build to qt 5.12.10

fix "Mismatching new/free or malloc/delete" found by CodeQL. (#649)

Create codeql-analysis.yml (#648)

* Create codeql-analysis.yml

* Update codeql-analysis.yml

ozi: don't deinit a write stream if we don't have one.

We can get into close() with a null stream_ TextStream. I think it has to do with ozi_openfile() reopening a file with a new name to match an extension of a certain type so it's renaming a file that hasn't been written. See issue #646

update build to Qt 5.15.1 from 5.15.0 (#642)

eliminate duplicate ignrando test. (#641)

backward compatible changes for Qt6. (#640)

* backward compatible changes for Qt6.

This is mostly QRegExp -> QRegularExpression.
Some QRegExp usage was was replaced by QString::trimmed.
There is also a QNetworkRequest redirect attribute change that is backward
compatible to Qt5.9.

* eliminate usage of unnecessary regular expressions.

* bring serial_unix into Qt6 compliance.

QSet::toList is gone, QSet::values is identical.
qSort is deprecated.

add ability to disable map preview in GUI (#635)

* add CONFIG option to disable map preview feature.

* additional excludes for disabled map preview

Fix potential segmentation faults with exif. (#639)

1. A pointer exif_app_ to an ExifApp on QList<ExifApp> exif_apps was saved.
After this the list was modified.  This can lead to the saved pointer becoming
invalid.
2. The ExifApp structure had a dtor, but no other special functions, i.e.  it
violated the rule of 3 and the rule of 5.  Operations on QList<ExifApp> may
cause an ExifApp on the list to be copied or destroyed.  If an ExifApp is
destroyed, then the ExifApp dtor would close the files, even though there could
be a copy of the ExifApp expecting the files still to be open.

This scenerio occured with Qt6, causing segmentation faults in exif.test.